Skip to content

fix: configure already existing wind-tunnel repo's github pages settings#68

Merged
mattyg merged 1 commit intomainfrom
fix/wind-tunnel-gh-pages
Feb 11, 2026
Merged

fix: configure already existing wind-tunnel repo's github pages settings#68
mattyg merged 1 commit intomainfrom
fix/wind-tunnel-gh-pages

Conversation

@mattyg
Copy link
Member

@mattyg mattyg commented Feb 11, 2026 •
edited by coderabbitai bot
Loading

This was causing errors on pulumi deployments.

Summary by CodeRabbit

  • Chores
    • Configured GitHub Pages deployment settings to use the gh-pages branch as the deployment source.

@cocogitto-bot
Copy link

cocogitto-bot bot commented Feb 11, 2026

✔️ b357315 - Conventional commits check succeeded.

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026 •
edited
Loading

Walkthrough

Adds a Pages configuration section to the wind-tunnel repository arguments, specifying GitHub Pages deployment with source branch "gh-pages" and root path "/". This modification configures repository settings without affecting the core repository creation logic.

Changes

Cohort / File(s) Summary
GitHub Pages Configuration
main.go		 Adds Pages configuration to wind-tunnel repository arguments, setting source branch to "gh-pages" and path to "/" for GitHub Pages deployment.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • jost-s
  • ThetaSinner
  • cdunster
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding GitHub Pages configuration to the wind-tunnel repository. It is specific, concise, and clearly reflects the core modification in the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/wind-tunnel-gh-pages

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Feb 11, 2026

🍹 preview on holochain/holochain/github

Pulumi report

View in Pulumi Cloud

  Previewing update (holochain/github)

View Live: https://app.pulumi.com/holochain/holochain/github/previews/965ba36a-a442-42fa-b04b-f0980c6bf6ad

pulumi:pulumi:Stack: (same)
  [urn=urn:pulumi:github::holochain::pulumi:pulumi:Stack::holochain-github]
  ~ github:index/repository:Repository: (update)
      [id=wind-tunnel]
      [urn=urn:pulumi:github::holochain::github:index/repository:Repository::wind-tunnel]
      [provider=urn:pulumi:github::holochain::pulumi:providers:github::default_6_11_0::dce38a79-6242-43ba-a5ff-26fae2b6f93c]
    + pages: {
        + buildType : "legacy"
        + source    : {
            + branch    : "gh-pages"
            + path      : "/"
          }
      }
  + github:index/actionsSecret:ActionsSecret: (create)
      [urn=urn:pulumi:github::holochain::github:index/actionsSecret:ActionsSecret::kitsune2-cachix-auth-token]
      [provider=urn:pulumi:github::holochain::pulumi:providers:github::default_6_11_0::dce38a79-6242-43ba-a5ff-26fae2b6f93c]
      destroyOnDrift: true
      plaintextValue: [secret]
      repository    : "kitsune2"
      secretName    : "CACHIX_AUTH_TOKEN"
Resources:
  + 1 to create
  ~ 1 to update
  2 changes. 362 unchanged

@mattyg mattyg requested a review from a team February 11, 2026 18:19
coderabbitai[bot]
coderabbitai bot reviewed Feb 11, 2026
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents 
In `@main.go`:
- Around line 113-118: The Pages and Source fields are using value types but the
Pulumi GitHub SDK expects pointer types: update windTunnelRepositoryArgs.Pages
to be a pointer to github.RepositoryPagesArgs and set its Source to a pointer to
github.RepositoryPagesSourceArgs (i.e., use &github.RepositoryPagesArgs{...} and
&github.RepositoryPagesSourceArgs{...}) so the types match the SDK signatures
for RepositoryArgs.Pages and RepositoryPagesArgs.Source.

@mattyg mattyg marked this pull request as draft February 11, 2026 18:23
@mattyg mattyg removed the request for review from a team February 11, 2026 18:23
@mattyg mattyg marked this pull request as ready for review February 11, 2026 18:36
@mattyg mattyg requested a review from a team February 11, 2026 18:37
@mattyg mattyg merged commit 0bb3f6f into main Feb 11, 2026
4 checks passed
@mattyg mattyg deleted the fix/wind-tunnel-gh-pages branch February 11, 2026 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@jost-s jost-s jost-s approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mattyg @jost-s